-
-
Notifications
You must be signed in to change notification settings - Fork 484
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mrp_sale_info: Materialize link between sale order lines and created MOs #1353
base: 16.0
Are you sure you want to change the base?
mrp_sale_info: Materialize link between sale order lines and created MOs #1353
Conversation
Improve test with 3 steps delivery route and use RecordCapturer
77cd0be
to
d7e675d
Compare
@grindtildeath @sebalix @ajaniszewska-dev @JuMiSanAr @pedrobaeza maybe you want to have a look since you worked in the linked PRs? I just edited a bit the commit messages in order to follow https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has provoked is done headaches, as the compute is launched on update time
Thanks for the review!
I guess that's the usual behavior when a new stored computed field is added, why is this case different? |
I mean that you should provide a init hook or auto_init override for pre-creating the DB schema and fast-populate data for existing records. |
Thanks for the hint, so at upgrade time the computed fields should be populated with an optimized calculation (you mean like SQL queries?) instead of letting the field compute itself, did I understand that correctly? |
Correct. |
I tried to do a SQL query for computing all the values in |
Forward port of #1298 and #1337.